Skip to content

Replace deprecated row/cell/instance terminology with "item" and "value" terms#4075

Merged
Keavon merged 1 commit intomasterfrom
table-terminology
Apr 29, 2026
Merged

Replace deprecated row/cell/instance terminology with "item" and "value" terms#4075
Keavon merged 1 commit intomasterfrom
table-terminology

Conversation

@Keavon
Copy link
Copy Markdown
Member

@Keavon Keavon commented Apr 29, 2026

Partly closes #3779.

Copy link
Copy Markdown
Contributor

@cubic-dev-ai cubic-dev-ai Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No issues found across 49 files

Confidence score: 5/5

  • Automated review surfaced no issues in the provided summaries.
  • No files require special attention.

Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request implements a comprehensive terminology refactor, replacing terms like "row", "instance", and "collection" with "item", "occurrence", and "list" to better reflect the Table data structure's usage. It also standardizes the capitalization of UI panel names in documentation and renames various internal variables and methods (e.g., instance_path to tree_path and per_glyph_instances to per_glyph_items). Review feedback highlights several instances where long comment lines should be wrapped for better readability and suggests reverting a logic simplification that negatively impacted code clarity.

Comment thread node-graph/libraries/wgpu-executor/src/texture_conversion.rs
Comment thread node-graph/nodes/graphic/src/graphic.rs
Comment thread node-graph/nodes/graphic/src/graphic.rs
@Keavon Keavon merged commit 1be9676 into master Apr 29, 2026
10 of 11 checks passed
@Keavon Keavon deleted the table-terminology branch April 29, 2026 02:13
oluseyi pushed a commit to oluseyi/Graphite that referenced this pull request Apr 29, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Data trees

1 participant